Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: by default hide cancel #696

Merged
merged 1 commit into from
Oct 22, 2022

Conversation

SwaySway
Copy link
Contributor

@SwaySway SwaySway commented Oct 12, 2022

Issue #, if available:
asana task: https://app.asana.com/0/1202185248783861/1203024227380062/f
Description of changes:

  • hide cancel component generation by editing cta matrix
  • hide onCancel variable and type via shouldIncludeCancel helper function

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@SwaySway SwaySway requested a review from a team as a code owner October 12, 2022 00:18
@codecov-commenter
Copy link

codecov-commenter commented Oct 12, 2022

Codecov Report

Merging #696 (4df3534) into tagged-release/cms-support (7152ffb) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                      Coverage Diff                       @@
##           tagged-release/cms-support     #696      +/-   ##
==============================================================
+ Coverage                       91.86%   91.89%   +0.02%     
==============================================================
  Files                             110      111       +1     
  Lines                            4069     4082      +13     
  Branches                         1125     1130       +5     
==============================================================
+ Hits                             3738     3751      +13     
  Misses                            310      310              
  Partials                           21       21              
Impacted Files Coverage Δ
.../codegen-ui-react/lib/forms/react-form-renderer.ts 82.46% <ø> (ø)
.../forms/form-renderer-helper/event-handler-props.ts 100.00% <100.00%> (ø)
...eact/lib/forms/form-renderer-helper/type-helper.ts 100.00% <100.00%> (ø)
...ui/lib/generate-form-definition/helpers/map-cta.ts 93.33% <100.00%> (+0.47%) ⬆️
packages/codegen-ui/lib/utils/form-utils.ts 100.00% <100.00%> (ø)
packages/codegen-ui/lib/utils/index.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7152ffb...4df3534. Read the comment docs.

cwoolum
cwoolum previously approved these changes Oct 12, 2022
@SwaySway SwaySway force-pushed the fix/default-hide-cancel branch from 80d3990 to 7152ffb Compare October 21, 2022 23:27
@SwaySway SwaySway changed the base branch from tagged-release/forms-and-views to tagged-release/cms-support October 21, 2022 23:30
@SwaySway SwaySway force-pushed the fix/default-hide-cancel branch from af1cffe to 4df3534 Compare October 21, 2022 23:53
@SwaySway SwaySway merged commit 662dd10 into tagged-release/cms-support Oct 22, 2022
@SwaySway SwaySway deleted the fix/default-hide-cancel branch October 22, 2022 00:12
zchenwei added a commit that referenced this pull request Nov 17, 2022
* feat: init form support for has-one relationship (#718)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* chore: wrap relationship fields in ArrayField (#719)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* chore: split up form-renderer-helper (#720)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* chore: map props on relationship-based Autocomplete (#721)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* feat: add relationship suggestion query to forms (#722)

* feat: add relationship suggestion query

* chore: map props on relationship-based Autocomplete (#721)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* feat: add relationship suggestion query

* fix: remove extra declarations

Co-authored-by: Scott Young <scoyou@amazon.com>
Co-authored-by: Hein Jeong <73264629+hein-j@users.noreply.github.com>
Co-authored-by: Hein Jeong <heinje@amazon.com>

* fix: hide cancel button by default (#696)

* chore: modify ArrayField to support has-one relationships (#724)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* chore: support not-model HAS_ONE relationships (#726)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* chore: map type file for model fields (#727)

Co-authored-by: Hein Jeong <heinje@amazon.com>

* feat: add handling for excluded sectional elements

* fix: cli test fix

* fix: handle association breaking changes for new schema

* Add support for "belongsTo" relationships for create forms (#755)

* feat: wip belongs to support

* feat: belongsTo working for create form

* chore: add golden file for belongsTo create form

* fix: arrayfield/autocomplete bugs

* chore: update snapshots

* chore: handle review comments

* chore: add tests and update snapshots

* feat: adding many to many support for form creation

* test: fix tests

* test: fix test

* feat: adding many-to-many relationship create form golden file

* chore: update create file

* test: fix test

* test: fix unit tests

* test: fix tests

* fix: fix merge conflicts

* test: fix timestamp type

* chore: add tag update form golden file with manyToMany relationship

Co-authored-by: Hein Jeong <73264629+hein-j@users.noreply.github.com>
Co-authored-by: Hein Jeong <heinje@amazon.com>
Co-authored-by: Scott Young <snyoung@gmail.com>
Co-authored-by: Scott Young <scoyou@amazon.com>
Co-authored-by: Josue Ruiz <7465495+SwaySway@users.noreply.github.com>
Co-authored-by: Roshane Pascual <rotp@amazon.com>
Co-authored-by: Brandon Lyons <lyonsbp@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants